Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Refactor] Switch 컴포넌트에 토큰 적용 #34

Merged
merged 5 commits into from
Jun 2, 2024

Conversation

ghdtjgus76
Copy link
Collaborator

🎉 변경 사항

Switch 컴포넌트에 토큰 적용했습니다.

🚩 관련 이슈

🙏 여기는 꼭 봐주세요!

theme에 정의되어 있는 radius랑 spacing은 기존에 정의된대로 사용하려면 아래처럼 써야 하더라구요

borderRadius: 'radius.full'

radius를 반복하지 않고 쓰는 게 더 좋을 거 같기도 하고 radius, spacing을 제외한 다른 토큰들이 그렇게 되어 있기도 해서 아래와 같이 쓸 수 있도록 radius, spacing 객체 구조를 변경했습니다.

borderRadius: 'full'

Copy link
Contributor

Copy link
Collaborator

@SeieunYoo SeieunYoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

전 좋습니다 🙌 토큰이 수정돼서 styled-system 에서도 변경이 생기지 않나유?

Copy link
Member

@hamo-o hamo-o left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저도 좋아요!!
생각해 보니.. 저도 r값을 토큰을 이용하도록 수정해야겠네요..!

Copy link
Contributor

github-actions bot commented Jun 2, 2024

@ghdtjgus76
Copy link
Collaborator Author

전 좋습니다 🙌 토큰이 수정돼서 styled-system 에서도 변경이 생기지 않나유?

넵 wow-ui styled-system에 반영해뒀고 왜인지는 모르겠지만 루트에 styled-system 폴더가 있길래 같이 삭제했어요!

@ghdtjgus76 ghdtjgus76 merged commit df1dffe into main Jun 2, 2024
3 checks passed
@ghdtjgus76 ghdtjgus76 deleted the refactor/switch-component-token branch June 2, 2024 04:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants